Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[stable13] Log full exception in cron instead of only the message #7820

Merged
merged 1 commit into from
Jan 12, 2018
Merged

[stable13] Log full exception in cron instead of only the message #7820

merged 1 commit into from
Jan 12, 2018

Conversation

MorrisJobke
Copy link
Member

@MorrisJobke MorrisJobke commented Jan 12, 2018 •
edited
Loading

Edit: This somehow was merged against master:

Backport of #7818

@MorrisJobke
Log full exception in cron instead of only the message
7bc1001 
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke MorrisJobke added the 3. to review Waiting for reviews label Jan 12, 2018
@MorrisJobke MorrisJobke added this to the Nextcloud 13 milestone Jan 12, 2018
@MorrisJobke MorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 12, 2018
@codecov
Copy link

codecov bot commented Jan 12, 2018 •
edited
Loading

Codecov Report

Merging #7820 into master will not change coverage.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master    #7820   +/-   ##
=========================================
  Coverage     51.23%   51.23%           
  Complexity    24971    24971           
=========================================
  Files          1607     1607           
  Lines         95008    95008           
  Branches       1376     1376           
=========================================
  Hits          48674    48674           
  Misses        46334    46334
Impacted Files Coverage Δ Complexity Δ
cron.php 0% <0%> (ø) 0 <0> (ø) ⬇️

@MorrisJobke MorrisJobke merged commit b6e4f34 into master Jan 12, 2018
@MorrisJobke MorrisJobke deleted the 13-7818 branch January 12, 2018 16:58
@MorrisJobke MorrisJobke mentioned this pull request Jun 21, 2018
Closed
@MorrisJobke MorrisJobke removed this from the Nextcloud 13 milestone Jun 21, 2018
@MorrisJobke MorrisJobke mentioned this pull request Jun 21, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rullzer rullzer rullzer approved these changes

@nickvergessen nickvergessen nickvergessen approved these changes

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst

Assignees
No one assigned
Labels
4. to release Ready to be released and/or waiting for tests to finish
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MorrisJobke @rullzer @nickvergessen